Skip to content

Conversation

@mrnicegyu11
Copy link
Member

@mrnicegyu11 mrnicegyu11 commented Nov 26, 2025

What do these changes do?

Add prometheus metric scraping for: vector, loki, tempo, grafana, jaeger

Bonus:

  • Disable telemetry sending on tempo
  • Fix shellcheck pre-commit hook failure on a random bash-script

tested this on osparc.local

Related issue/s

#1279

Related PR/s

https://git.speag.com/oSparc/osparc-ops-deployment-configuration/-/merge_requests/1692

Checklist

  • I tested and it works

mrnicegyu11 and others added 30 commits October 15, 2024 16:18
Merge remote-tracking branch 'upstream/main'
…oundation#979)

* Introduce longhorn chart

* Further longhorn configuration

* Longhorn: further settings configuration

* Fix longhorn configuration bugs

Extra: introduce longhorn pv vales for portainer

* Add comment for deletion longhorn

* Further longhorn configuration

* Add README.md for Longhorn wit FAQ

* Update Longhorn readme

* Update readme

* Futher LH configuration

* Update LH's Readme

* Update Longhorn Readme

* Improve LH's Readme

* LH: Reduce reserved default disk space to 5%

Since we use a dedicated disk for LH, we can go ahead with 5%

* Use values to set Longhorn storage class

* Update LH's Readme

* LH Readme: add requirements reference

* PR Review: bring back portainer s3 pv

* LH: decrease portinaer volume size
Merge remote-tracking branch 'upstream/main'
@mrnicegyu11 mrnicegyu11 added this to the Imparable milestone Nov 26, 2025
@mrnicegyu11 mrnicegyu11 self-assigned this Nov 26, 2025
@mrnicegyu11 mrnicegyu11 added observability alerting/monitoring FAST labels Nov 26, 2025
@mrnicegyu11 mrnicegyu11 changed the title Add prometheus metric scraping for: vector, loki, tempo, grafana, jaeger 🎨 Add prometheus metrics: vector, loki, tempo, grafana, jaeger Nov 26, 2025
@mrnicegyu11 mrnicegyu11 marked this pull request as ready for review November 26, 2025 09:33
Copy link
Collaborator

@YuryHrytsuk YuryHrytsuk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks! Left comments

Copy link
Contributor

@bisgaard-itis bisgaard-itis left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cool, thanks a lot 👍🏻

Copy link
Contributor

@bisgaard-itis bisgaard-itis left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Very nice. Thanks a lot for the effort. 👍🏻

@YuryHrytsuk YuryHrytsuk self-requested a review November 27, 2025 12:09
Copy link
Collaborator

@YuryHrytsuk YuryHrytsuk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks 🙏

@mrnicegyu11 mrnicegyu11 enabled auto-merge (squash) December 1, 2025 08:46
@mrnicegyu11 mrnicegyu11 merged commit fc6694a into ITISFoundation:main Dec 1, 2025
3 checks passed
@mrnicegyu11
Copy link
Member Author

works
image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

FAST observability alerting/monitoring

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants